Skip to content

[8.4] Add polyfill for ReflectionConstant #524

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 24, 2025

Conversation

DanielEScherzer
Copy link
Contributor

No description provided.

@DanielEScherzer DanielEScherzer changed the title Add polyfill for ReflectionConstant [8.4] Add polyfill for ReflectionConstant May 4, 2025
@DanielEScherzer
Copy link
Contributor Author

All failures and errors are unrelated in the PHPUnit tests (mostly mbstring and Grapheme related)

@Ayesh
Copy link
Contributor

Ayesh commented May 5, 2025

Wow, I didn't think it will be possible to polyfill ReflectionConstant, but great job pulling it off!

@OskarStark OskarStark changed the title [8.4] Add polyfill for ReflectionConstant [8.4] Add polyfill for ReflectionConstant May 14, 2025
@DanielEScherzer DanielEScherzer force-pushed the ReflectionConstant branch 3 times, most recently from d2351b5 to 610bba8 Compare May 14, 2025 14:27
@DanielEScherzer
Copy link
Contributor Author

@fabpot is there anything I can do to move this forward?

@DanielEScherzer DanielEScherzer requested a review from stof June 22, 2025 22:52
@nicolas-grekas nicolas-grekas force-pushed the ReflectionConstant branch 2 times, most recently from 7243e0d to 2d152f8 Compare June 24, 2025 13:02
@nicolas-grekas
Copy link
Member

Thank you @DanielEScherzer.

@nicolas-grekas nicolas-grekas merged commit 8e9fcc5 into symfony:1.x Jun 24, 2025
2 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants